Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Claude finished @julienrbrt's task in 1m 25s —— View job

Code Review

• Gather context and analyze PR changes
• Review go.mod changes
• Provide final assessment

Summary

This PR re-adds local replace directives to apps/evm/go.mod and apps/testapp/go.mod for development, following the pattern already present in apps/grpc/go.mod. The previous bump commit (#3160) had commented them out; this restores them as active replace blocks.

Changes Reviewed

apps/evm/go.mod — Adds:

replace (
    github.com/evstack/ev-node => ../../
    github.com/evstack/ev-node/execution/evm => ../../execution/evm
)

apps/testapp/go.mod — Adds:

replace github.com/evstack/ev-node => ../../.

Observations

Correctness: The replace directives are valid relative paths within this monorepo and are consistent with the existing pattern in apps/grpc/go.mod. Nothing suspicious here.

Minor inconsistency (non-blocking): apps/testapp/go.mod uses ../../. while apps/evm/go.mod uses ../../. These are functionally equivalent, but normalizing to ../../ (without the trailing dot) would be slightly cleaner for consistency.

Missing core replace: Neither apps/evm nor apps/testapp adds a replace for github.com/evstack/ev-node/core, even though it's a direct dependency in both. However, this matches the pattern in apps/grpc/go.mod, so it appears intentional — core likely doesn't need local replacement in the current dev workflow.

go.sum cleanup: The corresponding entries were removed from go.sum files as expected when switching from resolved versions to local paths.

Verdict

The changes are straightforward and correct. This follows the established monorepo development pattern. LGTM.

The latest Buf updates on your PR. Results from workflow CI / buf-check (pull_request).

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 60.20%. Comparing base (d6bbca6) to head (3efc8c2).
⚠️ Report is 2 commits behind head on main.

@@           Coverage Diff           @@
##             main    #3161   +/-   ##
=======================================
  Coverage   60.20%   60.20%           
=======================================
  Files         115      115           
  Lines       11896    11896           
=======================================
  Hits         7162     7162           
  Misses       3923     3923           
  Partials      811      811           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.